Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FE 테스트 라이브러리 설정 #78

Merged
merged 5 commits into from
Nov 7, 2024
Merged

Conversation

parkblo
Copy link
Collaborator

@parkblo parkblo commented Nov 6, 2024

✏️ 한 줄 설명

이 PR의 주요 변경 사항이나 구현된 내용을 간략히 설명해 주세요.

FE 테스트 라이브러리가 실행 가능하도록 설정했습니다.

✅ 작업 내용

  • Storybook 설치 및 설정
  • Vitest 설치
  • Playwright 설치

🏷️ 관련 이슈

📸 스크린샷/영상

이번 PR에서 변경되거나 추가된 뷰가 있는 경우 이미지나 동작 영상을 첨부해 주세요.

스크린샷 2024-11-06 오후 6 48 51

  • 스토리북이 정상 작동되는 화면
  • 스크린샷 내 표시된 컴포넌트들은 스토리북에서 기본 제공해주는 것. 삭제하고 커밋하였음.

📌 리뷰 진행 시 참고 사항

리뷰 코멘트 작성 시 특정 사실에 대해 짚는 것이 아니라 코드에 대한 의견을 제안할 경우, 강도를 함께 제시해주세요! (1점: 가볍게 참고해봐도 좋을듯 ↔ 5점: 꼭 바꾸는 게 좋을 것 같음!)

스크린샷 2024-11-06 오후 6 14 22

  1. 새로운 라이브러리를 설치할 시, pnpm-lock.yaml에서 기존 라이브러리의 version 항목도 변경이 일어났는데, 찾아본 결과 의존성 업데이트라고 하여 큰 문제가 없다고 판단했습니다. 따라서 커밋에 변경을 포함시켰습니다.

  2. 각 테스트 라이브러리에서 추가적으로 설정하면 좋을 점들에 대해 리뷰남겨주시면 반영하도록 하겠습니다!

@parkblo parkblo added this to the FE 환경 설정 milestone Nov 6, 2024
@parkblo parkblo self-assigned this Nov 6, 2024
@parkblo parkblo requested a review from a team as a code owner November 6, 2024 15:04
@parkblo parkblo requested review from heegenie and fru1tworld and removed request for a team November 6, 2024 15:04
@fru1tworld
Copy link
Collaborator

LGTM

Copy link
Collaborator

@heegenie heegenie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 테스트를 위한 설정은 일단 이정도로 마무리해도 괜찮을 것 같습니다 👍

@hoqn hoqn linked an issue Nov 7, 2024 that may be closed by this pull request
@hoqn hoqn merged commit 48d1e10 into dev-fe Nov 7, 2024
3 checks passed
@heegenie heegenie deleted the feature/fe-test-lib-init branch November 8, 2024 02:07
@heegenie heegenie restored the feature/fe-test-lib-init branch November 8, 2024 02:11
@heegenie heegenie deleted the feature/fe-test-lib-init branch November 8, 2024 02:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

테스트 라이브러리를 설정한다.
4 participants